Skip to content

[WPE] WPE Platform: handle multiple next presentation update callbacks#44931

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
carlosgcampos:wpe-platform-multiple-next-update-cbs
May 6, 2025
Merged

[WPE] WPE Platform: handle multiple next presentation update callbacks#44931
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
carlosgcampos:wpe-platform-multiple-next-update-cbs

Conversation

@carlosgcampos
Copy link
Copy Markdown
Contributor

@carlosgcampos carlosgcampos commented May 5, 2025

48d151b

[WPE] WPE Platform: handle multiple next presentation update callbacks
https://bugs.webkit.org/show_bug.cgi?id=292533

Reviewed by Nikolas Zimmermann.

We currently have a limit of 1, but some layout tests end up calling
callAfterNextPresentationUpdate before the previous one is completed, so
we can just handle multiple callbacks like gtk port does.

* Source/WebKit/UIProcess/API/wpe/WPEWebViewPlatform.cpp:
(WKWPE::ViewPlatform::~ViewPlatform):
(WKWPE::ViewPlatform::dispatchPendingNextPresentationUpdateCallbacks):
(WKWPE::ViewPlatform::callAfterNextPresentationUpdate):
* Source/WebKit/UIProcess/API/wpe/WPEWebViewPlatform.h:

Canonical link: https://commits.webkit.org/294553@main

51a4f1b

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 win
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 ✅ 🧪 win-tests
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ✅ 🛠 wpe-cairo
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🛠 gtk
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🧪 api-gtk
✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2 ✅ 🛠 playstation
✅ 🛠 🧪 unsafe-merge ✅ 🛠 tv ✅ 🛠 mac-safer-cpp
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@carlosgcampos carlosgcampos requested a review from a team as a code owner May 5, 2025 12:22
@carlosgcampos carlosgcampos self-assigned this May 5, 2025
@carlosgcampos carlosgcampos added the WPE WebKit WebKit WPE component label May 5, 2025
@carlosgcampos carlosgcampos added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label May 6, 2025
https://bugs.webkit.org/show_bug.cgi?id=292533

Reviewed by Nikolas Zimmermann.

We currently have a limit of 1, but some layout tests end up calling
callAfterNextPresentationUpdate before the previous one is completed, so
we can just handle multiple callbacks like gtk port does.

* Source/WebKit/UIProcess/API/wpe/WPEWebViewPlatform.cpp:
(WKWPE::ViewPlatform::~ViewPlatform):
(WKWPE::ViewPlatform::dispatchPendingNextPresentationUpdateCallbacks):
(WKWPE::ViewPlatform::callAfterNextPresentationUpdate):
* Source/WebKit/UIProcess/API/wpe/WPEWebViewPlatform.h:

Canonical link: https://commits.webkit.org/294553@main
@webkit-commit-queue webkit-commit-queue force-pushed the wpe-platform-multiple-next-update-cbs branch from 51a4f1b to 48d151b Compare May 6, 2025 08:42
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 294553@main (48d151b): https://commits.webkit.org/294553@main

Reviewed commits have been landed. Closing PR #44931 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 48d151b into WebKit:main May 6, 2025
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label May 6, 2025
@carlosgcampos carlosgcampos deleted the wpe-platform-multiple-next-update-cbs branch May 6, 2025 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WPE WebKit WebKit WPE component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants